변경된 코드 리뷰 방향 PR 템플릿에 반영 #1124
Merged
변경된 코드 리뷰 방향 PR 템플릿에 반영 #1124
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
There was a problem hiding this comment.
혹시 다음 사람이 이전 사람 pr에 본인을 리뷰어로 지정해도 괜찮을까요?
왜냐하면 기존에 순서를 바꾸는 목적이 이전 사람의 역할이이 다음 사람 pr을 올리는 시점에 영향을 안 받게 하기 위함이였기 때문입니다! 예를 들어 월요일에 가용시간이 많고 토요일에 시간이 없으신 분이 있다면 다음 분이 토요일에 pr올리게 됐을 때 요구사항을 충족하지� 못할 가능성이 높다고 생각합니다. ! 그래서 pr을 올리는 사람이 올리는 시점에 모든 역할을 수행할 수 있게 하는게 이상적이라고 생각합니다! 물론 병합이나 pr에 대한 답변 자체 리뷰 시간에 영향을 받겠지만, 리뷰와 리뷰어 지정 자체는 다음 사람이 하는 방향이 기존 설계한 방향에 일관적일 것 같습니다!
SamTheKorean
approved these changes
Mar 29, 2025
.github/pull_request_template.md
Outdated
| @@ -14,6 +14,6 @@ | |||
|
|
|||
| - [ ] 우측 메뉴에서 PR을 **Projects**에 추가해주세요. | |||
| - [ ] **Projects**의 오른쪽 버튼(▼)을 눌러 확장한 뒤, **Week**를 현재 주차로 설정해주세요. | |||
| - [ ] 바로 앞에 PR을 열어주신 분을 코드 검토자로 지정해주세요. | |||
| - [ ] 바로 다음에 PR을 열어주신 분을 코드 검토자로 지정해주세요. | |||
There was a problem hiding this comment.
코멘트에서 언급한 설계 방향의 일관성을 위해서는 "바로 이전 PR에 본인을 코드 리뷰어로 지정해주세요"가 더 적합하다고 생각합니다!
There was a problem hiding this comment.
"바로 이전에 올라온 PR에 본인을 코드 리뷰어로 지정해주세요" 라고 수정했는데 어떠신가요?
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
위키에 코드 리뷰 가이드와 PR 템플릿의 내용이 상이한 부분이 있습니다. PR 작성자가 PR 템플릿 대로 리뷰어를 지정하면, PR 검토자 입장에서는 예전처럼 바로 앞에 올라온 PR이 아닌 바로 뒤에 올라온 PR을 리뷰하게 됩니다.
혼선을 방지하기 위해서 4기부터 변경되는 코드 리뷰 방향을 PR 템플릿에 반영합니다.